Skip to content

Conversation

@andy-stark-redis
Copy link
Contributor

@andy-stark-redis andy-stark-redis commented Oct 29, 2024

DOC-4367

I'll make a separate PR to add the release notes to the RDI repo when we're happy with them.

@andy-stark-redis andy-stark-redis requested a review from a team October 29, 2024 10:28
@andy-stark-redis andy-stark-redis self-assigned this Oct 29, 2024
- Installation on [Kubernetes]({{< relref "/integrate/redis-data-integration/installation/install-k8s" >}}) using a [Helm chart](https://helm.sh/docs/). You can install on [OpenShift](https://docs.openshift.com/) or other flavours of K8s using Helm.

- Improvements for installation on VMs:
- Installer checks if the OS firewall is enabled on Ubuntu and RHEL.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What is it linked to?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The links are to the Helm docs and the OpenShift docs. Would you prefer to remove the links?

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think they need to linking just the particular words or not have them at all

- Improved verification of RDI installation.
- Installer verifies if the RDI database is in use by another instance of RDI.
- Installer checks and warns if any [`iptables`](https://www.netfilter.org/projects/iptables/index.html) rules are set.
- Improved message when RDI tries to connect to its Redis database with invalid TLS keys.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

what is it linked to?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Links to the iptable home page. Again, I'll remove it if you think it's better without it.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

the link needs to be just on the word IPtables

## Limitations

- RDI can write data to a Redis Active-Active database. However, it doesn't support writing data to two or more Active-Active replicas. Writing data from RDI to several Active-Active replicas could easily harm data integrity as RDI is not synchronous with the source database commits.
- RDI write-behind (which is currently in preview) should not be used on the same data set that RDI ingest is writing to Redis. This would either cause an infinite loop or would harm the data integrity, since both ingest and write-behind are asynchronous, eventually-consistent processes.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Remove write-behind. We already removed it to another section in docs and about to stop selling it in few days

when you start a pipeline from the CLI tool. This means that you must
provide the operator with a service account that has permissions to create and manipulate pods in the namespace.
You can use the RDI [Helm chart](https://helm.sh/docs/topics/charts/) to install
on [Kubernetes (K8s)](https://kubernetes.io/). This creates:
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

it's good to explicitly mention Openshift as most customers will use it
something like on Kubernetes (K8s) including Red Hat Openshift

- Installation on [Kubernetes]({{< relref "/integrate/redis-data-integration/installation/install-k8s" >}}) using a [Helm chart](https://helm.sh/docs/). You can install on [OpenShift](https://docs.openshift.com/) or other flavours of K8s using Helm.

- Improvements for installation on VMs:
- Installer checks if the OS firewall is enabled on Ubuntu and RHEL.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think they need to linking just the particular words or not have them at all

- Improved verification of RDI installation.
- Installer verifies if the RDI database is in use by another instance of RDI.
- Installer checks and warns if any [`iptables`](https://www.netfilter.org/projects/iptables/index.html) rules are set.
- Improved message when RDI tries to connect to its Redis database with invalid TLS keys.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

the link needs to be just on the word IPtables

1. Start the installation:

```bash
helm install <The logical chart name> rdi-k8s/<rdi-tag>/helm
Copy link
Contributor

@galilev galilev Oct 29, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

please add --create-namespace -n rdi at the end of the helm install command

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done.

global:
imagePullSecrets: []
# - name: "image-pull-secret"
# or
Copy link
Contributor

@galilev galilev Oct 30, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

remove lines 78-79. :

 imagePullSecrets: []
 # - name: "image-pull-secret"


 image:
   registry: docker.io
   repository: redis

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done.

@andy-stark-redis andy-stark-redis merged commit af02ec7 into main Oct 30, 2024
5 checks passed
@andy-stark-redis andy-stark-redis deleted the DOC-4367-k8s-docs branch October 30, 2024 16:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants